Skip to content

ref(store): Validate database_name before CREATE DATABASE#721

Merged
untitaker merged 2 commits into
mainfrom
ref/validate-database-name
Jun 19, 2026
Merged

ref(store): Validate database_name before CREATE DATABASE#721
untitaker merged 2 commits into
mainfrom
ref/validate-database-name

Conversation

@untitaker

@untitaker untitaker commented Jun 17, 2026

Copy link
Copy Markdown
Member

CREATE DATABASE cannot use bind parameters for the database name, so the name has to be interpolated directly into the statement. This restricts it to a safe identifier charset (ASCII alphanumerics and underscores) before interpolation, ruling out SQL injection via database_name.

This PR mainly exists to pacify security scanners, not because this is a real issue.

ref STREAM-1125

🤖 Generated with Claude Code

`CREATE DATABASE` cannot use bind parameters for the database name, so
the name is interpolated into the statement. Restrict it to a safe
identifier charset (ASCII alphanumerics and underscores) to rule out SQL
injection.

ref STREAM-1125

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@untitaker untitaker requested a review from a team as a code owner June 17, 2026 23:45
@linear-code

linear-code Bot commented Jun 17, 2026

Copy link
Copy Markdown

STREAM-1125

@untitaker untitaker merged commit 95f0e16 into main Jun 19, 2026
26 checks passed
@untitaker untitaker deleted the ref/validate-database-name branch June 19, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants